-
Notifications
You must be signed in to change notification settings - Fork 138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added fix for upstream pipeline failure. #3504
base: master
Are you sure you want to change the base?
Conversation
611c876
to
25415ef
Compare
--ansible-module-path tests/dogtag/pytest-ansible/common-modules \ | ||
--junit-xml pki_cert_scep_1664435_junit.xml \ | ||
-qsvvv \ | ||
tests/dogtag/pytest-ansible/pytest/ca/scep/test_bug_1664435_pki_ca_scep_enrollment.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to merge this into the previous step (lines 100-110) so we don't have to keep modifying the workflow whenever we add new tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this as per my previous job (https://github.com/dogtagpki/pki/pull/3351/files)
Should I add jobs as below?
-qsvvv \
tests/dogtag/pytest-ansible/pytest/sanity/test_role_users.py
tests/dogtag/pytest-ansible/pytest/ca/scep/test_bug_1664435_pki_ca_scep_enrollment.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm thinking the SCEP test should be added into tests/dogtag/pytest-ansible/pytest/sanity/test_role_users.py
so that if we want to run sanity test manually we only need to run just that one file (assuming SCEP test is part of sanity test).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
f4ff01e
to
2d824f6
Compare
@gswami90 Hi, I've fixed the missing pip3 issue. Could you rebase the PR? Thanks. |
…pki/-/pipelines/291988444 Signed-off-by: Gaurav Swami <[email protected]>
2d824f6
to
87356f5
Compare
Added fix for upstream pipeline failure https://gitlab.com/dogtagpki/pki/-/pipelines/291988444
Signed-off-by: Gaurav Swami [email protected]